Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Miscellaneous cleanup needed for first release #38

Merged
merged 20 commits into from
Dec 13, 2024
Merged

Miscellaneous cleanup needed for first release #38

merged 20 commits into from
Dec 13, 2024

Conversation

dylanhmorris
Copy link
Collaborator

No description provided.

@dylanhmorris dylanhmorris changed the title Miscellaneous cleanup needed for first stable release Miscellaneous cleanup needed for first release Dec 13, 2024
@dylanhmorris
Message = FALSE in flu scoring vignette
0aaff11
@dylanhmorris
harmonize a few more argument names
1e6ea9e
@dylanhmorris
Run roxygen
d312cf5
@dylanhmorris
One more argument name harmonization
db7bf04
@dylanhmorris
Make testing setup compatible with CI, remove namespace qualification…
772bc8c 
… in test scripts
@dylanhmorris
more test cleanup
3321cd5
@dylanhmorris
Stronger assertions for date validation
edc0cb8
@dylanhmorris
Fix column name
396a4e0
@dylanhmorris
null date handling
a33ea05
@dylanhmorris
assignments are not recognized as assertions in some contexts
fe1c32c
@dylanhmorris
Rename another plotting function
4fa87f8
@dylanhmorris
Run roxygen
46c6ad1
@dylanhmorris
Remove superfluous date cast
f98a45b
@dylanhmorris
Update y transform argument name in vignette
cf1dfc4
@dylanhmorris
Further simplify tests in epiweek_to_date
dc35b65
@dylanhmorris
Very basic test for daily_to_epiweekly
9834469
@dylanhmorris
Add test of daily to epiweekly error for duplicates
fe827f2
@dylanhmorris
Add test for partial week filtering in daily to epiweekly
0c82190
@dylanhmorris dylanhmorris marked this pull request as ready for review December 13, 2024 15:53
@dylanhmorris dylanhmorris enabled auto-merge (squash) December 13, 2024 15:53
@dylanhmorris dylanhmorris merged commit 656226b into main Dec 13, 2024
9 checks passed
@dylanhmorris dylanhmorris deleted the dhm-misc-cleanup branch December 13, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dylanhmorris